-
Notifications
You must be signed in to change notification settings - Fork 143
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor diff_drive demo #635
base: ros2
Are you sure you want to change the base?
Conversation
Signed-off-by: Carlos Agüero <[email protected]>
Signed-off-by: Carlos Agüero <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we are not publishing the tf
from Gazebo.
My suggestion here is to add the tf to the bridge
- ros_topic_name: "tf"
gz_topic_name: "/model/vehicle_green/tf"
ros_type_name: "tf2_msgs/msg/TFMessage"
gz_type_name: "gz.msgs.Pose_V"
direction: GZ_TO_ROS
- ros_topic_name: "tf"
gz_topic_name: "/model/vehicle_blue/tf"
ros_type_name: "tf2_msgs/msg/TFMessage"
gz_type_name: "gz.msgs.Pose_V"
direction: GZ_TO_ROS
and a static transform for each vehicle:
<node pkg="tf2_ros" exec="static_transform_publisher" args="--frame-id world --child-frame-id vehicle_green/odom" />
<node pkg="tf2_ros" exec="static_transform_publisher" args="--frame-id world --child-frame-id vehicle_blue/odom" />
And modify the RViz file to sue the "Fixed frame" as world
and add the other odometry marker for the green vehicle
ros_gz_sim_demos/README.md
Outdated
|
||
ros2 topic pub /model/vehicle_blue/cmd_vel geometry_msgs/msg/Twist "{linear: {x: 5.0}, angular: {z: 0.5}}" | ||
|
||
This demo also shows the use of custom QoS parameters. The commands are |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do you remove this comment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The latest version of the code (79ece9e) preserves that block.
Signed-off-by: Carlos Agüero <[email protected]>
Thanks for the suggestion. Added in 80d120b. Could you give it a try please? |
Refactors the
diff_drive
demo leveraging the latest improvements inros_gz
.Summary
Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-by
messages.